Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add network_policy to google_container_cluster #159

Merged
merged 4 commits into from
Jun 5, 2019
Merged

Conversation

aaron-lane
Copy link
Contributor

@aaron-lane aaron-lane commented Jun 3, 2019

This branch adds an explicit network_policy block to the google_container_cluster, as recommended by hashicorp/terraform-provider-google#3673. This solves the issue of the partially converged suites which obviates the need for a second converge.

@aaron-lane aaron-lane added the bug Something isn't working label Jun 3, 2019
@aaron-lane aaron-lane force-pushed the aaron-lane-fix-ci branch 2 times, most recently from 117d58b to 39b40fc Compare June 4, 2019 21:15
@aaron-lane aaron-lane force-pushed the aaron-lane-fix-ci branch from 39b40fc to 063e70b Compare June 5, 2019 15:04
@aaron-lane aaron-lane marked this pull request as ready for review June 5, 2019 15:05
@aaron-lane aaron-lane added the enhancement New feature or request label Jun 5, 2019
Copy link
Contributor

@morgante morgante left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's have a more descriptive PR title and sensible provider default.

autogen/variables.tf Outdated Show resolved Hide resolved
aaron-lane added a commit that referenced this pull request Jun 5, 2019
@aaron-lane aaron-lane changed the title Fix CI issues Add network_policy to google_container_cluster Jun 5, 2019
@aaron-lane aaron-lane merged commit b40ddea into master Jun 5, 2019
@aaron-lane aaron-lane deleted the aaron-lane-fix-ci branch June 5, 2019 19:28
aaron-lane added a commit that referenced this pull request Jun 5, 2019
…changelog

[skip ci] Add CHANGELOG entry for #159.
CPL-markus pushed a commit to WALTER-GROUP/terraform-google-kubernetes-engine that referenced this pull request Jul 15, 2024
CPL-markus pushed a commit to WALTER-GROUP/terraform-google-kubernetes-engine that referenced this pull request Jul 15, 2024
…-modules/aaron-lane-fix-ci

Add network_policy to google_container_cluster
CPL-markus pushed a commit to WALTER-GROUP/terraform-google-kubernetes-engine that referenced this pull request Jul 15, 2024
…-modules/aaron-lane-159-changelog

[skip ci] Add CHANGELOG entry for terraform-google-modules#159.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants